-
Notifications
You must be signed in to change notification settings - Fork 227
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add typos
to CI
#441
Add typos
to CI
#441
Conversation
@TheDan64, this PR bundles the adding |
@@ -0,0 +1,9 @@ | |||
[default] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's with the underscore prefix? Shouldn't it be .typos.toml?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is so that the typos utility treats it as a configuration file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Glad to help! |
I added the
typos
utility to CI, and simplified the enum variants in theBuilderError
from #436 .Description
I added the
typos
utility to CI (and an exclude file).I also removed the
GEPError
as it did not make sense and the 2 othersGEPPointee
andGEPIndex
fit better. The 2 lines which usedGEPError
have been replaced to use more the descriptive variants.Related Issue
This is related to #438.
It also resolves what @marcospb19 raised in #436.
How This Has Been Tested
CI, including my new addition to it, is passing.
Checklist